Motivation

We previously defaulted the HTTPClient to .init(), but that's not correct as it was never getting shut down.

Modifications

Instead of creating a new client, just introduce our own shared one and use that as the default value.

Once AHC provides a shared client, we can default to that in the configuration initializer.

Result

No more crashing clients on dealloc.

Test Plan

All tests pass.